Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support relaunch debug protocol #181

Merged
merged 21 commits into from
Jan 31, 2024
Merged

Conversation

Christian-Holbrook
Copy link
Collaborator

Persist the debug session for debug protocol. This is accomplished by parsing the telnet logs for the line "Waiting for debugging connection" and then creating the control port. When the application is closed, we continue to parse the telnet logs for that line. When that line is detected the control port will be created again.

A new signal app-ready is emitted to let BrightScriptDebugSession know the debug adapters are ready. Both adapters have to emit this signal.

When the app is closed via a home key press, reset the breakpoint managers state for the debug protocol.

Copy link
Member

@TwitchBronBron TwitchBronBron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fe

src/adapters/DebugProtocolAdapter.ts Outdated Show resolved Hide resolved
src/adapters/DebugProtocolAdapter.ts Outdated Show resolved Hide resolved
src/debugProtocol/client/DebugProtocolClient.ts Outdated Show resolved Hide resolved
src/adapters/DebugProtocolAdapter.ts Outdated Show resolved Hide resolved
src/adapters/DebugProtocolAdapter.ts Outdated Show resolved Hide resolved
Copy link
Member

@TwitchBronBron TwitchBronBron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few changes requested, but looking great!

Copy link
Member

@TwitchBronBron TwitchBronBron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested this, everything seems to be working properly. Thanks!

@TwitchBronBron TwitchBronBron merged commit e2b2ec9 into master Jan 31, 2024
6 checks passed
@TwitchBronBron TwitchBronBron deleted the support-relaunch-debug-protocol branch January 31, 2024 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants